Skip to content

Add DLOCKSS_NODE_NAME environment variable#723

Merged
physikerwelt merged 1 commit intomainfrom
DLOCKS_env
Feb 24, 2026
Merged

Add DLOCKSS_NODE_NAME environment variable#723
physikerwelt merged 1 commit intomainfrom
DLOCKS_env

Conversation

@physikerwelt
Copy link
Copy Markdown
Member

@physikerwelt physikerwelt commented Feb 24, 2026 •
edited by coderabbitai bot
Loading

  • Configuration update:
    • Added the DLOCKSS_NODE_NAME environment variable with the value MaRDI4NFDI to the service configuration in docker-compose-extra.yml.

Summary by CodeRabbit

  • Chores
    • Updated service configuration to enhance deployment setup.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 24, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f5c9a4f and 6b071e3.

📒 Files selected for processing (1)
  • docker-compose-extra.yml
📝 Walkthrough

Walkthrough

Adds a new environment variable DLOCKSS_NODE_NAME with value MaRDI4NFDI to the dlockss-node service configuration in docker-compose-extra.yml. No structural or behavioral changes are introduced.

Changes

Cohort / File(s) Summary
Environment Configuration
docker-compose-extra.yml		 Added DLOCKSS_NODE_NAME environment variable to the dlockss-node service.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A node gets its name with a line so fine,
MaRDI4NFDI—how it will shine!
In compose files where services align,
One variable added, the config's divine.
🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add DLOCKSS_NODE_NAME environment variable' directly and clearly describes the main change: adding a new environment variable to the configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch DLOCKS_env

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@physikerwelt physikerwelt merged commit 856ab7a into main Feb 24, 2026
1 check passed
@physikerwelt physikerwelt deleted the DLOCKS_env branch February 24, 2026 16:09
@coderabbitai coderabbitai bot mentioned this pull request Mar 2, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@physikerwelt